[ASAN] tests: Fix some memleaks in libarchive importer
authorColin Walters <walters@verbum.org>
Fri, 18 Nov 2016 15:32:59 +0000 (10:32 -0500)
committerAtomic Bot <atomic-devel@projectatomic.io>
Mon, 21 Nov 2016 16:34:06 +0000 (16:34 +0000)
Caught by `-fsanitize=address`.

Closes: #587
Approved by: jlebon

tests/test-libarchive-import.c

index ad2e4c94be36785b2190d4b6413de45621902601..05c5568d34f4b5a65c00cdbb0ac6fdbab15933b0 100644 (file)
@@ -468,6 +468,8 @@ entry_pathname_test_helper (gconstpointer data, gboolean on)
       goto out;
     }
 
+  archive_read_free (a);
+  ostree_repo_commit_modifier_unref (modifier);
  out:
   g_assert_no_error (error);
 }
@@ -534,6 +536,7 @@ test_libarchive_selinux (gconstpointer data)
   g_assert_cmpstr (buf, ==, "system_u:object_r:etc_t:s0");
 
  out:
+  archive_read_free (a);
   if (modifier)
     ostree_repo_commit_modifier_unref (modifier);
   g_assert_no_error (error);
@@ -562,5 +565,6 @@ int main (int argc, char **argv)
 
   if (td.tmpd && g_getenv ("TEST_SKIP_CLEANUP") == NULL)
     (void) glnx_shutil_rm_rf_at (AT_FDCWD, td.tmpd, NULL, NULL);
+  g_free (td.tmpd);
   return r;
 }